Skip to content
This repository was archived by the owner on Nov 17, 2017. It is now read-only.

Conversation

lmacko
Copy link
Contributor

@lmacko lmacko commented Sep 23, 2013

No description provided.

@bleathem
Copy link
Member

I made some grammatical corrections and merged into the branch "chart-docs":
https://github.com/richfaces/richfaces-docs/tree/chart-docs

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It allows to handle fired events using client-side JavaScript or server-side listener.

I't not clear what "It" is here. The previous tag you mentioned? or charts altogether?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I meant the chart component in general.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Lukindo, I updated the chart-docs branch to more clearly reflect your intention.

@bleathem
Copy link
Member

These docs are a great start. It identifies the relevant tags and provides some "getting started" examples. Further documentation examples can get use-case specific and provide some real-world examples.

@bleathem
Copy link
Member

@lfryc this pull request looks good to merge from my perspective (merge from the chart-docs branch). Do you have any additional comments before we proceed with the merge?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants